-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: enable datadog tracer #3431
Conversation
Signed-off-by: sawadashota <shota@sslife.tech>
@@ -6,7 +6,7 @@ WORKDIR /go/src/github.com/ory/hydra | |||
RUN mkdir -p ./internal/httpclient | |||
|
|||
COPY go.mod go.sum ./ | |||
COPY internal/httpclient/go.* ./internal/httpclient | |||
COPY internal/httpclient/go.* ./internal/httpclient/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without a trailing slash, it will fail.
Step 6/21 : COPY internal/httpclient/go.* ./internal/httpclient
When using COPY with more than one source file, the destination must be a directory and end with a /
make: *** [docker] Error 1
Seems like tests are failing, can you PTAL? :) |
Codecov Report
@@ Coverage Diff @@
## master #3431 +/- ##
==========================================
+ Coverage 76.47% 76.74% +0.26%
==========================================
Files 123 123
Lines 10535 9022 -1513
==========================================
- Hits 8057 6924 -1133
+ Misses 1990 1657 -333
+ Partials 488 441 -47
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@aeneasr I'm sorry, but I've fixed it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sawadashota
Load OpenTelemetry configuration to set up datadog tracer.
Related issue(s)
none
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security vulnerability, I
confirm that I got the approval (please contact
security@ory.sh) from the maintainers to push
the changes.
works.
Further Comments